Skip to content

Include OID explicitly in the SELECT clause for older Postgres databases #680

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Include OID explicitly in the SELECT clause for older Postgres databases #680

wants to merge 1 commit into from
+1 −1

Conversation

HyunSangHan
Copy link
Contributor

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You use the code formatters provided here and have them applied to your changes. Don't submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

Issue description

  • This change addresses a compatibility issue with PostgreSQL 11.
  • In PostgreSQL 11, the oid column is considered a system column and is not included by default in SELECT * queries. As a result, the query must explicitly specify the oid field to retrieve it. This change updates the query to ensure the oid is selected explicitly for proper behavior across PostgreSQL versions.

New Public APIs

None.

Additional context

  • No other functional changes were introduced.

@injae-kim injae-kim mentioned this pull request May 18, 2025
Closed
@mp911de mp911de added the type: bug A general bug label Oct 6, 2025
@mp911de mp911de changed the title GH-670 Include the OID explicitly in the SELECT clause Include OID explicitly in the SELECT clause for older Postgres databases Oct 6, 2025
@mp911de mp911de added this to the 1.0.8.RELEASE milestone Oct 6, 2025
@mp911de mp911de closed this in f3752eb Oct 6, 2025
mp911de added a commit that referenced this pull request Oct 6, 2025
@mp911de
Polishing.
a5d3b06 
Refine tests to ensure proper timing out.

[#680][#670]
mp911de pushed a commit that referenced this pull request Oct 6, 2025
@HyunSangHan @mp911de
Include OID explicitly in the SELECT clause for older Postgres da…
0b7471a 
…tabases.

[resolves #680][resolves #670]
mp911de added a commit that referenced this pull request Oct 6, 2025
@mp911de
Polishing.
8fa9ea3 
Refine tests to ensure proper timing out.

[#680][#670]
@mp911de
Copy link
Collaborator

mp911de commented Oct 6, 2025

Thank you for your contribution. That's merged and backported now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
type: bug A general bug
Projects
None yet
Milestone
1.0.8.RELEASE
Development

Successfully merging this pull request may close these issues.

PostgresTypes lookup fails on Postgres 11
2 participants
@HyunSangHan @mp911de